Skip to content

docs: correct rust workspace crate count to 11#3269

Open
nankingjing wants to merge 1 commit into
ultraworkers:mainfrom
nankingjing:contrib/rust-crate-count
Open

docs: correct rust workspace crate count to 11#3269
nankingjing wants to merge 1 commit into
ultraworkers:mainfrom
nankingjing:contrib/rust-crate-count

Conversation

@nankingjing

Copy link
Copy Markdown

Summary

Follow-up doc fix after #3268.

The Stats section of rust/README.md states 9 crates in the workspace, but the workspace (members = ["crates/*"]) now contains 11 crates:

api, claw-analog, claw-rag-service, commands, compat-harness, mock-anthropic-service, plugins, runtime, rusty-claude-cli, telemetry, tools

(verified by counting rust/crates/*/Cargo.toml — 11 manifests). This PR updates only that number; it does not restructure the layout or responsibilities sections.

Note on a second candidate fix (not included)

While investigating I also checked the reported broken link in how_to_run.md ([futute.md](futute.md)). I did not change it: neither futute.md nor a correctly-spelled future.md exists anywhere in the repository, so retargeting the link would not resolve to a real file. Left untouched pending a decision on whether that target doc should be created.

@1716775457damn

Copy link
Copy Markdown

Good catch — docs that say one thing while the workspace defines another are a real source of confusion for new contributors. With 11 crates now, are you planning to document what the 11th is separately or leave it self-explanatory from the workspace layout?

@nankingjing

Copy link
Copy Markdown
Author

The workspace uses members = ["crates/*"] so any directory dropped under rust/crates/ automatically becomes a member — there isn't really a single 11th crate, the count just drifted from 9 to 11 over time. All 11 are listed in the PR description (api, claw-analog, claw-rag-service, commands, compat-harness, mock-anthropic-service, plugins, runtime, rusty-claude-cli, telemetry, tools). I figured that plus the glob pattern makes it self-explanatory. Happy to add a one-liner listing them in the README if you think it'd help though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants